Skip to content

ostest/hrtimer: align hrtimer test and add threaded hrtimer tests #3347

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wangchdo wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ostest/hrtimer: align hrtimer test and add threaded hrtimer tests #3347

wangchdo wants to merge 2 commits into from
+101 −9

Conversation

@wangchdo
Copy link
Contributor

@wangchdo wangchdo commented Jan 20, 2026 •
edited
Loading

Summary

  1. align hrtimer callback declaration with latest update of hrtimer
  2. Add threaded hrtimer tests

Impact

Hritmer ostest improvements, no impact to any other functions

apache/nuttx#18025 rely on this PR to fix the CI build issue

Testing

ostest passed with hrtimer enabled:

rv-virt:smp64

NuttShell (NSH)
nsh> 
nsh> uname -a
NuttX 0.0.0 3485d61db2-dirty Jan 20 2026 11:42:50 risc-v rv-virt
nsh> 
nsh> ostest

(...)

user_main: hrtimer test

End of test memory usage:
VARIABLE  BEFORE   AFTER
======== ======== ========
arena     1fc0a00  1fc0a00
ordblks         9       11
mxordblk  1f71c68  1f50cd0
uordblks    15c90    27320
fordblks  1faad70  1f996e0

Final memory usage:
VARIABLE  BEFORE   AFTER
======== ======== ========
arena     1fc0a00  1fc0a00
ordblks         1       11
mxordblk  1fb5908  1f50cd0
uordblks     b0f8    27320
fordblks  1fb5908  1f996e0
user_main: Exiting
ostest_main: Exiting with status 0
nsh>

@wangchdo wangchdo mentioned this pull request Jan 20, 2026
Open
@xiaoxiang781216
Copy link
Contributor

xiaoxiang781216 commented Jan 20, 2026

@wangchdo please fix:

/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule

@wangchdo
ostest/hrtimer: align hrtimer callback declaration
5179bad 
  align hrtimer callback declaration with latest update of hrtimer

Signed-off-by: Chengdong Wang <[email protected]>
@wangchdo
ostest/hrtimer: Add threaded hrtimer tests
6749f3c 
  Add threaded hrtimer tests

Signed-off-by: Chengdong Wang <[email protected]>
@wangchdo wangchdo force-pushed the hrtimer_test_alignment branch from 2bd1730 to 6749f3c Compare January 20, 2026 09:25
@wangchdo
Copy link
Contributor Author

wangchdo commented Jan 20, 2026

@wangchdo please fix:

/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule
/github/workspace/sources/apps/Application.mk:238: target 'signest.c.github.workspace.sources.apps.testing.ostest.o' given more than once in the same rule

I rebased this PR, I think this problem should be fixed

@simbit18
Copy link
Contributor

simbit18 commented Jan 20, 2026

Hi @wangchdo, still not resolved.

====================================================================================
Configuration/Tool: qemu-armv8a/nsh_smp
2026-01-20 09:41:58
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
/d/a/nuttx-apps/nuttx-apps/sources/apps/Application.mk:238: target 'signest.c.d.a.nuttx-apps.nuttx-apps.sources.apps.testing.ostest.o' given more than once in the same rule
/d/a/nuttx-apps/nuttx-apps/sources/apps/Application.mk:238: target 'signest.c.d.a.nuttx-apps.nuttx-apps.sources.apps.testing.ostest.o' given more than once in the same rule
/d/a/nuttx-apps/nuttx-apps/sources/apps/Application.mk:238: target 'signest.c.d.a.nuttx-apps.nuttx-apps.sources.apps.testing.ostest.o' given more than once in the same rule
/d/a/nuttx-apps/nuttx-apps/sources/apps/Application.mk:238: target 'signest.c.d.a.nuttx-apps.nuttx-apps.sources.apps.testing.ostest.o' given more than once in the same rule
  [1/1] Normalize qemu-armv8a/nsh_smp
====================================================================================

@simbit18 simbit18 mentioned this pull request Jan 20, 2026
Open
@simbit18 simbit18 requested review from acassis and linguini1 January 20, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xiaoxiang781216 xiaoxiang781216 xiaoxiang781216 approved these changes

@acassis acassis Awaiting requested review from acassis

@linguini1 linguini1 Awaiting requested review from linguini1

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

Area: Testing Size: M

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@wangchdo @xiaoxiang781216 @simbit18